Factory agent pulls skills on demand instead of front-loading them#5537
Draft
jurgenwerk wants to merge 1 commit into
Draft
Factory agent pulls skills on demand instead of front-loading them#5537jurgenwerk wants to merge 1 commit into
jurgenwerk wants to merge 1 commit into
Conversation
Front-load only the factory workflow skill; every other skill appears as a one-line index entry in the system prompt and is fetched on demand through a new read_skill factory tool. Skill sources are now exactly two: .agents/skills-orchestrator (workflow) and boxel-cli plugin/skills (domain library). --skills-dir replaces the on-demand library with an operator-curated folder (fail-fast when empty). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The factory's per-issue agent used to receive a resolver-picked set of skills pasted in full into every system prompt, with a token budget that silently truncated whatever didn't fit. It now gets progressive disclosure:
.agents/skills-orchestrator/(factory workflow skills, still front-loaded) andpackages/boxel-cli/plugin/skills/(the domain library, on demand). The root.agents/skills/copies (boxel-development,ember-best-practices) are no longer read.realm-sync,file-ops,realm-history,profile,boxel-environment) are excluded from the index — their write commands would fight the orchestrator's workspace→realm sync — and the index preamble tells the agent not to run realm-mutatingboxelcommands skills mention.--skills-dir <path>replaces the on-demand library with an operator-curated folder: nothing bundled is indexed or readable, no exclusion filtering, fail-fast at startup if the folder yields no skills. Front-loaded workflow skills are unaffected.Verified on a staging run: the implementation-issue prompt shrank to one front-loaded skill plus an 18-line index, and the agent made 12 targeted
read_skillcalls (the core skill plus 11 of its 32 references, each pulled right before the work it covers) with all validators green in a single iteration.🤖 Generated with Claude Code